d4d7072d9f5652b65128709429e102035969c389,findbugs/src/java/edu/umd/cs/findbugs/SourceLineAnnotation.java,SourceLineAnnotation,writeXML,#XMLOutput#boolean#,592

Before Change


			packageName = classname.substring(0,1+classname.lastIndexOf('.'));
		XMLAttributeList attributeList = new XMLAttributeList()
			.addAttribute("classname", classname)
			.addAttribute("start", String.valueOf(getStartLine()))
			.addAttribute("end", String.valueOf(getEndLine()))
			.addAttribute("startBytecode", String.valueOf(getStartBytecode()))
			.addAttribute("endBytecode", String.valueOf(getEndBytecode()));

After Change


			packageName = classname.substring(0,1+classname.lastIndexOf('.'));
		XMLAttributeList attributeList = new XMLAttributeList()
			.addAttribute("classname", classname);
		int n = getStartLine(); // start/end are now optional (were too many "-1"s in the xml)
		if (n >= 0) attributeList.addAttribute("start", String.valueOf(n));
		n = getEndLine();
		if (n >= 0) attributeList.addAttribute("end", String.valueOf(n));
		n = getStartBytecode(); // startBytecode/endBytecode haven't been set for a while now